Skip to content

Office Overview Part 1/5 - DAV search service, templates API and category providers#5683

Draft
moodyjmz wants to merge 33 commits into
feat/office-overview-v2from
feat/overview-pr1-categories-imports
Draft

Office Overview Part 1/5 - DAV search service, templates API and category providers#5683
moodyjmz wants to merge 33 commits into
feat/office-overview-v2from
feat/overview-pr1-categories-imports

Conversation

@moodyjmz
Copy link
Copy Markdown
Contributor

Part of the Office Overview feature — merge into feat/office-overview-v2.

  • DAV SEARCH service to fetch all office files by MIME type
  • Templates API to load creators/categories from the server
  • Category provider wiring to map MIME types to nav entries

elzody added 29 commits May 20, 2026 18:50
Adds the office overview page to the application bar, and the appropriate route, controller, and template

Signed-off-by: Elizabeth Danzberger <[email protected]>
Adds the OfficeOverview Vue component and renders it

Signed-off-by: Elizabeth Danzberger <[email protected]>
These template response parameters are used to determine where the app navigation and content are, so that keyboard users can jump to them easily

Signed-off-by: Elizabeth Danzberger <[email protected]>
The OfficeOverview Vue component needs to be rendered directly to `#content` else it results in a weird double-margin visual bug. It causes some styles to be applied twice. This is also consistent with other apps.

Signed-off-by: Elizabeth Danzberger <[email protected]>
We had imported material design icons for the office file type icons in the navigation bar, but it might be best to re-use the icons we are using in the rest of the richdocuments app now for consistency.

Signed-off-by: Elizabeth Danzberger <[email protected]>
Signed-off-by: Elizabeth Danzberger <[email protected]>
Adds some basic Cypress and PHP unit tests to have some tests in place already

Signed-off-by: Elizabeth Danzberger <[email protected]>
Fetch files via WebDAV and cache them, and using the entries in the navigation bar to filter them. Unfortunately we cannot use the files entry grid from the Files app, so we have to use a somewhat custom Vue component.

Signed-off-by: Elizabeth Danzberger <[email protected]>
Previously, fileid was used, but I learned that it is deprecated and snowflake IDs (node.id) should be used instead

Signed-off-by: Elizabeth Danzberger <[email protected]>
In order for the viewer to be initiated and usable for opening documents, we need to dispatch this event. Previously, `LoadAdditionalScripts` was used, but it actually does not do what is necessary. It is unrelated.

Signed-off-by: Elizabeth Danzberger <[email protected]>
Switch to using the event dispatcher instead of the service locator pattern. This also fixes a PHP Unit test

Signed-off-by: Elizabeth Danzberger <[email protected]>
Turns out CI does not load the viewer app, and even stubs won't work; it keeps throwing errors. So, we don't check that the event is dispatched now. We can test this in other ways, naturally, via E2E tests

Signed-off-by: Elizabeth Danzberger <[email protected]>
Seemed like a lot of extra steps to get the viewer loaded in the CI environment for the PHP tests, so we can just check if the class exists and gracefully skip it. The tests don't need it for now, and in a real scenario or E2E tests it will get dispatched just fine.

Signed-off-by: Elizabeth Danzberger <[email protected]>
For right now, I want to focus solely on traditional office file types. Later on it will be easy to add other types.

Signed-off-by: Elizabeth Danzberger <[email protected]>
Move to a custom Vue component for displaying the document cards.

Signed-off-by: Elizabeth Danzberger <[email protected]>
We can get rid of the OfficeFileEntry component because it was just acting as a wrapper around each FileCard. We can just use FileCard directly and move those methods to the OfficeOverview component

Signed-off-by: Elizabeth Danzberger <[email protected]>
Signed-off-by: Elizabeth Danzberger <[email protected]>
Inject an initial state that determines whether file previews should be shown or not, based on either core settings (enable_previews) or richdocuments settings.

Signed-off-by: Elizabeth Danzberger <[email protected]>
When previews are disabled, swap to using the file type icons instead.

Signed-off-by: Elizabeth Danzberger <[email protected]>
Signed-off-by: Elizabeth Danzberger <[email protected]>
Add Cypress assertion to check if the preview exists, and a PHP unit test to make sure the initial state is set correctly.

Signed-off-by: Elizabeth Danzberger <[email protected]>
A lot of the logic was redundant, so we can use a setup method to not have to redefine the mocked objects

Signed-off-by: Elizabeth Danzberger <[email protected]>
Make them more like file entry grid by removing the center alignment of the text.

Signed-off-by: Elizabeth Danzberger <[email protected]>
This was missing and is an office type, so should be added.

Signed-off-by: Elizabeth Danzberger <[email protected]>
Adds a search bar so that the current view can be searched to easily find files

Signed-off-by: Elizabeth Danzberger <[email protected]>
Adds tests for the search functionality

Signed-off-by: Elizabeth Danzberger <[email protected]>
The card should give the impression that it is "clickable"

Signed-off-by: Elizabeth Danzberger <[email protected]>
DAV search finds documents that are not only in the root folder but also recursively in subdirectories. It was specified that it should be used in the feature spec anyway.

Signed-off-by: Elizabeth Danzberger <[email protected]>
@moodyjmz moodyjmz force-pushed the feat/overview-pr1-categories-imports branch from 54eb244 to 0886176 Compare May 20, 2026 20:16
moodyjmz and others added 4 commits May 21, 2026 09:23
Uses sortNodes from @nextcloud/files with sortFavoritesFirst and
mtime descending so the most recently edited files appear first with
starred items pinned to the top.

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
Signed-off-by: James Manuel <[email protected]>
…rrel imports

Nav items are now driven by whatever TemplateFileCreator objects are
registered with the NC Files API, so installing or removing an office
suite automatically updates the sidebar. Uses a single fetch on mount
for both the category list and the MIME filter for DAV SEARCH.

Shows NcEmptyContent when no providers are installed rather than
displaying an empty nav with no files.

Signed-off-by: James Manuel <[email protected]>
@moodyjmz moodyjmz force-pushed the feat/overview-pr1-categories-imports branch from 0886176 to 33949fd Compare May 21, 2026 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants